-
Couldn't load subscription status.
- Fork 471
Hint about partial application in error where required labels are missing #7807
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
rescript
@rescript/darwin-arm64
@rescript/darwin-x64
@rescript/linux-arm64
@rescript/linux-x64
@rescript/runtime
@rescript/win32-x64
commit: |
|
|
||
| - Missing arguments that must be provided: ~d, ~c, ~b | ||
| - Missing arguments that must be provided: ~d, ~c, ~b | ||
| - Hint: Did you want to partially apply the function? You can do that by putting `...` just before the closing parens of the function call |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be possible to give a code example using the actual function and args the user tried to apply?
- Hint: Did you want to partially apply the function?
You can do that by putting `...` just before the closing parens of the function call:
f(v, ...)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Definitely! Just problematic if the function application is large, but maybe it's fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy with that 👍
d33d948 to
c5bd1ff
Compare
Just a small tweak. Happy for feedback on the hint message itself.
I'll add a changelog just before merging.